Skip to content

Newrelic integration#13

Open
kshitij-partech wants to merge 17 commits intohtdcx:mainfrom
punchh:newrelic-integration
Open

Newrelic integration#13
kshitij-partech wants to merge 17 commits intohtdcx:mainfrom
punchh:newrelic-integration

Conversation

@kshitij-partech
Copy link
Copy Markdown

No description provided.

vivekpunchh and others added 17 commits April 11, 2025 13:14
…ecret Manager (#2)

* [CX-2682, CX-2684] - Move all configuration/secrets to AWS Secret Manager

* CX-2682: Addressed code review changes.

* CX-2682: Removed unwanted files.

* CX-2683: Fix the code vulnerabilities/snyk reported issues

* [CX-2682] Redis TTL & Ping endpoint.

* CX-2682: Addressed code review changes.
This pull request modifies the `CheckUpdate` function in
`request/client.go` to change the update behavior. Specifically, it
updates the `IsMandatory` field to be set to `false` instead of `true`
for both `updateInfoRedis` and `updateInfo` objects.

Key change:

*
[`request/client.go`](diffhunk://#diff-3f9ae5f6e0bc02077798851f584d54a1df6d7d64338b01141fc78e85fef6ccf6L67-R67):
Updated the `IsMandatory` field in the `CheckUpdate` function to be set
to `false`, making updates non-mandatory.
[[1]](diffhunk://#diff-3f9ae5f6e0bc02077798851f584d54a1df6d7d64338b01141fc78e85fef6ccf6L67-R67)
[[2]](diffhunk://#diff-3f9ae5f6e0bc02077798851f584d54a1df6d7d64338b01141fc78e85fef6ccf6L107-R107)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants